-
Notifications
You must be signed in to change notification settings - Fork 18
Add Texture Guidelines #23
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Add Texture Guidelines #23
Conversation
DevGeniusCode
commented
Mar 11, 2025
- Closes Document rules for Texture Restoration and PSD layer names #21
@DevGeniusCode @xezon Is this ready to go? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I wrote one more section in #21 just from memory without checking much.
## Texture Naming Structure | ||
|
||
>[!TIP] | ||
> the `_en` and `_rn` textures invalid because 99% of textures with these suffixes do not have proper night lighting but |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
are invalid
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A few textual remarks. Please also rebase.
|
||
## PSD Layer Naming Conventions | ||
|
||
- If the Background layer is flattened, then it is the original retail texture (DDS) in best discovered quality. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
'then it is' -> it represents
## PSD Layer Naming Conventions | ||
|
||
- If the Background layer is flattened, then it is the original retail texture (DDS) in best discovered quality. | ||
- If the bottom layer is not the original retail texture (DDS), then it has a name that represents the source texture |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
'then it has a name that represents' -> 'it should be named to reflect'
name in sufficient detail. That can be a DDS or TGA or a processed (AI upscale) texture, for example | ||
`cbcnvstr01_s.dds realesr-general-wdn-x4v3`. Or a "merged" aka composited PSD image of the same (and never another!) | ||
texture set, for example `cbcnvstr01 merged`. | ||
- If PSD textures are modified, then their edits need to move to lower order textures that use higher order merged |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
'then their edits need to' -> edits must
'lower order' -> 'lower-order'
'higher order' -> 'higher-order'
texture variant. | ||
- Layers & Groups can be copied to other texture variants and are expected to be identical if they share the same name | ||
within the same texture set. | ||
- Groups meant to be used to assemble an alpha channel are named `_alpha` and will be hidden before saving the image |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
'will' -> 'must'
textures (Pristine -> Damages, Summer -> Winter, ...). | ||
- Layer & Group names are lower case and use common keywords such as "damage", "light", "barricades", "snow", or any | ||
other name that gives it meaning. | ||
- Layer & Group names may end on a texture variant suffix (`_d`, `_g`, `_e`, ...) which helps to identify the origin |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
may or must?
textures. | ||
|
||
2. **PSD**: When working with **PSD** files, maintain an organized layer structure and use clear names for each texture | ||
variant (e.g., `_d`, `_g`). Layers with an alpha channel should be named `_alpha` and hidden before saving the file. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
'should' -> 'must'
2. **PSD**: When working with **PSD** files, maintain an organized layer structure and use clear names for each texture | ||
variant (e.g., `_d`, `_g`). Layers with an alpha channel should be named `_alpha` and hidden before saving the file. | ||
|
||
3. **Saving and Conversion**: After editing, textures should not be manually exported as DDS files. This should be |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Suggestion:
- Saving and Conversion: Don’t manually export DDS files. Texture conversion should be automated by the build pipeline via Mod Builder to ensure optimal quality and file size.